Skip to content

Fix call to spotiflow#1

Open
guiwitz wants to merge 1 commit into
trackmate-sc:mainfrom
guiwitz:main
Open

Fix call to spotiflow#1
guiwitz wants to merge 1 commit into
trackmate-sc:mainfrom
guiwitz:main

Conversation

@guiwitz
Copy link
Copy Markdown

@guiwitz guiwitz commented Dec 15, 2025

It seems that the current call to spotify.cli doesn't works as expected at least on Mac/Linux. This is a suggested fix for that call. See https://forum.image.sc/t/trackmate-spotiflow-not-running/118137?u=guiwitz for more details.

I compiled an alternative plugin locally using this PR and it seems to work.

@guiwitz guiwitz changed the title Fix call to spotify Fix call to spotiflow Dec 15, 2025
@tinevez tinevez self-assigned this Dec 15, 2025
@tinevez tinevez added the bug Something isn't working label Dec 15, 2025
@tinevez
Copy link
Copy Markdown
Member

tinevez commented Dec 15, 2025

I think the problem is more general. Trackastra, using the same system, depends on this as well.
Let me investigate in the core artifact.

@guiwitz
Copy link
Copy Markdown
Author

guiwitz commented Dec 15, 2025

The difference of spotify compared to trackastra and cellpose is that the client is registered with a different name than the package itself. So while you can just call cellpose or trackastra from the terminal and this will launch their clients, for spotiflow you need spotiflow-predict. See here for trackastra and here for cellpose.

So either you use spotiflow-predict or you call the module.

@guiwitz
Copy link
Copy Markdown
Author

guiwitz commented Dec 15, 2025

I'm just not sure what was the intention behind going from spotiflow-predict to the client here.

@imagesc-bot
Copy link
Copy Markdown

This pull request has been mentioned on Image.sc Forum. There might be relevant details there:

https://forum.image.sc/t/trackmate-spotiflow-not-running/118137/4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants